Skip to content

New Crowdin updates - #325

Merged
samolego merged 5 commits into
masterfrom
l10n_master
Dec 2, 2025
Merged

New Crowdin updates#325
samolego merged 5 commits into
masterfrom
l10n_master

Conversation

@samolego

@samolego samolego commented Nov 8, 2025

Copy link
Copy Markdown
Owner

Summary by CodeRabbit

  • Chores
    • Portuguese (pt-PT): Updated user-facing text with full Portuguese translations, minor wording and spacing fixes, and a corrected format placement in an uninstall confirmation.
    • Turkish (tr-TR): Revised UI labels, log text, and descriptive copy for clarity and consistency.
    • Dutch (nl-NL): Replaced English wording with Dutch translations, including clearer privilege messages and updated pluralized/app-selection strings.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai

coderabbitai Bot commented Nov 8, 2025

Copy link
Copy Markdown
Contributor

Walkthrough

Updated only localized Android string resources for Portuguese (pt-PT), Turkish (tr-TR) and Dutch (nl-NL); keys unchanged, values adjusted for translations, minor spacing/format-argument fixes; no code, manifest, or API changes.

Changes

Cohort / File(s) Summary
Portuguese (pt-PT) Localization
app/src/main/res/values-pt-rPT/strings.xml
Replaced many string values with Portuguese (pt-PT) translations; preserved keys. Minor wording, spacing and format-argument placement fixes (e.g., are_you_sure_to_uninstall_apps, select_all_enabled).
Turkish (tr-TR) Localization
app/src/main/res/values-tr-rTR/strings.xml
Updated Turkish string literals (notably no_apps_found, logs, copy_logs, log_copied, commits_url_description, bloat_list_url_description); keys unchanged.
Dutch (nl-NL) Localization
app/src/main/res/values-nl-rNL/strings.xml
Updated multiple Dutch translations including badge_info, auth_required_description, plural messages (clear_selected_apps, num_selected_apps, success_uninstalled, success_reinstalled) and descriptive texts; keys unchanged.

Sequence Diagram(s)

(No sequence diagram — changes are localization-only and do not alter control flow.)

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

  • Focus points:
    • Verify correct placement and numbering of format arguments (e.g., %1$s, %d) in values-pt-rPT/strings.xml.
    • Check plural entries in values-nl-rNL/strings.xml for correct quantity handling.
    • Spot-check Turkish literal changes for unintended semantic shifts.

Files to review:

  • app/src/main/res/values-pt-rPT/strings.xml
  • app/src/main/res/values-tr-rTR/strings.xml
  • app/src/main/res/values-nl-rNL/strings.xml

Possibly related PRs

  • New Crowdin updates #322 — Overlapping edits to values-pt-rPT/strings.xml, including are_you_sure_to_uninstall_apps formatting and other pt-PT translations.
  • New Crowdin updates #289 — Related localization updates touching commits_url* and bloat_list_url* descriptions across locales.
  • New Crowdin updates #311 — Related Dutch translations for values-nl-rNL/strings.xml (plural and descriptive string updates).

Poem

🐇 Eu traduzo com pulos e alegria,
frases em pt, Türkçe e Nederlands — que magia!
Presets e logs dançam em novo som,
um coelho sorri: "Tudo pronto, bom trabalho, bom!" 🥕

Pre-merge checks and finishing touches

❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Title check ❓ Inconclusive The title 'New Crowdin updates' is generic and vague, failing to convey specific information about the changeset's primary purpose. Consider using a more descriptive title that specifies the localization changes, such as 'Update Portuguese, Turkish, and Dutch translations' or 'Localize strings for pt-PT, tr-TR, and nl-NL'.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch l10n_master

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between aabe52f and 6e3d105.

📒 Files selected for processing (1)
  • app/src/main/res/values-nl-rNL/strings.xml (5 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
  • GitHub Check: 🤖 Validate Android Build
🔇 Additional comments (1)
app/src/main/res/values-nl-rNL/strings.xml (1)

13-13: Remaining translations look good.

The other Dutch translations across badge_info, clear_selected_apps, num_selected_apps, success_uninstalled, canta_description, confirm_uninstall_description, and auth_required_description are properly localized with consistent singular/plural forms where applicable.

Also applies to: 22-22, 26-26, 30-30, 39-39, 52-52, 112-112


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 19a491b and 05a8399.

📒 Files selected for processing (1)
  • app/src/main/res/values-pt-rPT/strings.xml (3 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
  • GitHub Check: 🤖 Validate Android Build
🔇 Additional comments (1)
app/src/main/res/values-pt-rPT/strings.xml (1)

65-65: Portuguese translations look good overall.

The localization updates appear well-translated and consistent with the existing Portuguese strings in the file. The text choices are natural and contextually appropriate for the UI.

Also applies to: 68-68, 69-69, 70-70, 71-71, 73-73, 82-82, 102-102, 103-103, 105-105, 107-107, 110-110, 113-113, 114-114

<string name="grant_shizuku_permission_to_canta">Garantir permissão do Canta no shizuko</string>
<string name="shizuku_required">Necessário shizuko</string>
<string name="shizuku_requirement_description">Canta uses Shizuku to uninstall apps without requiring root access. Shizuku provides a secure way to access system-level SDKs.</string>
<string name="shizuku_requirement_description">Canta utiliza Shizuku para desinstalar aplicativos sem necessitar de acesso root. Shizuku fornece uma maneira segura de acederr SDKs no sistema.</string>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟡 Minor

Typo in Portuguese text on line 96.

The word "acederr" appears to be a typo and should be "aceder" (Portuguese verb meaning "to access").

Apply this diff to fix the typo:

-    <string name="shizuku_requirement_description">Canta utiliza Shizuku para desinstalar aplicativos sem necessitar de acesso root. Shizuku fornece uma maneira segura de acederr SDKs no sistema.</string>
+    <string name="shizuku_requirement_description">Canta utiliza Shizuku para desinstalar aplicativos sem necessitar de acesso root. Shizuku fornece uma maneira segura de aceder SDKs no sistema.</string>
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
<string name="shizuku_requirement_description">Canta utiliza Shizuku para desinstalar aplicativos sem necessitar de acesso root. Shizuku fornece uma maneira segura de acederr SDKs no sistema.</string>
<string name="shizuku_requirement_description">Canta utiliza Shizuku para desinstalar aplicativos sem necessitar de acesso root. Shizuku fornece uma maneira segura de aceder SDKs no sistema.</string>
🤖 Prompt for AI Agents
In app/src/main/res/values-pt-rPT/strings.xml around line 96, the Portuguese
string contains a typo: "acederr" should be corrected to "aceder"; update the
string value so it reads "Canta utiliza Shizuku para desinstalar aplicativos sem
necessitar de acesso root. Shizuku fornece uma maneira segura de aceder SDKs no
sistema." to fix the spelling error.

@samolego
samolego merged commit c03cde1 into master Dec 2, 2025
2 checks passed
@samolego
samolego deleted the l10n_master branch December 2, 2025 19:05
@coderabbitai coderabbitai Bot mentioned this pull request Dec 5, 2025
@coderabbitai coderabbitai Bot mentioned this pull request Jan 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants